-
Notifications
You must be signed in to change notification settings - Fork 7
added auth_log_severity field to the Buddy reply #629
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🐳 Docker Images PushedBuddy Test-Kit Images:
Base image used: These images contain the buddy code from this PR and can be used for testing. Built from commit: 96ddccb |
|
The latest change reflects the recent discussion at https://docs.google.com/document/d/12vAQgphEA5E5Xwp8xNE5J0CVySNm_5iJnVFPG-plcvU |
🐳 Docker Images PushedBuddy Test-Kit Images:
Base image used: These images contain the buddy code from this PR and can be used for testing. Built from commit: 1566192 |
sanikolaev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's improve the examples, so there's at least one which includes the new "log" functionality.
|
added example of the log object at the Buddy HTTP response at a51e304 |
clt❌ CLT tests in Failed tests:🔧 Edit failed tests in UI: test/clt-tests/http-interface/sql-endpoint.rec––– input –––
rm -f /var/log/manticore/searchd.log; stdbuf -oL searchd $SEARCHD_FLAGS > /dev/null; if timeout 10 grep -qm1 '\[BUDDY\] started' <(tail -n 1000 -f /var/log/manticore/searchd.log); then echo 'Buddy started!'; else echo 'Timeout or failed!'; cat /var/log/manticore/searchd.log;fi
––– output –––
OK
––– input –––
mysql -h0 -P9306 -e "CREATE TABLE t (id INT, value TEXT, value_attr STRING) min_infix_len = '3' min_prefix_len = '3'; INSERT INTO t VALUES (1, 'example', 'example'), (2, 'test', 'test');"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "select * from t limit 1"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "select * from t where match('test')"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "select count(*) from t"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "select * from t where match('exmaple') option fuzzy=1"
––– output –––
- {"took":0,"timed_out":false,"hits":{"total":1,"total_relation":"eq","hits":[{"_id":1,"_score":%{NUMBER},"_source":{"value":"example","value_attr":"example"}}]}}
+ {"error":"only SELECT queries are supported"}
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "create table t_copy like t"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "show tables"
––– output –––
OK
––– input –––
curl -s -X POST "http://localhost:9308/sql" -d "insert into t values (3, 'new', 'new')"
––– output –––
OK
––– input –––
curl -s "http://localhost:9308/sql?query=select%20count(*)%20from%20t"
––– output –––
OK |
🐳 Docker Images PushedBuddy Test-Kit Images:
Base image used: These images contain the buddy code from this PR and can be used for testing. Built from commit: a51e304 |
to allow error message from the reply log into auth.log by daemon